Skip to content

fix: support ossAuto#85

Merged
rsonghuster merged 1 commit intomasterfrom
oss-auto
Sep 24, 2025
Merged

fix: support ossAuto#85
rsonghuster merged 1 commit intomasterfrom
oss-auto

Conversation

@mozhou52
Copy link
Copy Markdown
Collaborator

support ossAuto

@mozhou52 mozhou52 force-pushed the oss-auto branch 4 times, most recently from 9c435f8 to d5cf679 Compare September 23, 2025 08:32
Comment thread src/subCommands/deploy/impl/function.ts Outdated
if (region === 'cn-heyuan-acdr-1') {
ossEndpoint = `oss-${region}-internal.aliyuncs.com`;
}
if (process.env.FC_CODE_TEMP_OSS_ENDPOINT) {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

这个应该不需要吧

Comment thread src/subCommands/deploy/impl/function.ts
Comment thread src/resources/oss/index.ts Outdated
securityToken: credentials.SecurityToken,
endpoint: ossEndpoint,
regionId: region,
readTimeout: 60000,
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

只有 readTimeout 有 connectTimeout 吗? 这个最好也调大一些

@mozhou52 mozhou52 force-pushed the oss-auto branch 2 times, most recently from 7b8fcef to 29259c5 Compare September 24, 2025 02:08
@mozhou52 mozhou52 force-pushed the oss-auto branch 3 times, most recently from 2ecaad9 to 6f6b444 Compare September 24, 2025 07:12
@rsonghuster rsonghuster merged commit 5b0a410 into master Sep 24, 2025
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants